Skip to content

fix: grant FilOzzy admin access to filecoin-pin - #211

Merged
rjan90 merged 1 commit into
filecoin-project:masterfrom
rjan90:phi/add-filozzy-filecoin-pin-admin
Jun 11, 2026
Merged

fix: grant FilOzzy admin access to filecoin-pin#211
rjan90 merged 1 commit into
filecoin-project:masterfrom
rjan90:phi/add-filozzy-filecoin-pin-admin

Conversation

@rjan90

@rjan90 rjan90 commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Grant FilOzzy admin access on filecoin-project/filecoin-pin.
  • Move FilOzzy from push access to admin access.

Ref: FilOzone/tpm-utils#16 (comment)

@github-actions

Copy link
Copy Markdown
Contributor

The following access changes will be introduced as a result of applying the plan:

Access Changes
User filozzy:
  - will have the permission to filecoin-pin change from push to admin
User juliangruber:
  - will have the permission to filecoin-pin change from push to admin

@rjan90
rjan90 marked this pull request as ready for review June 11, 2026 07:31
@rjan90
rjan90 requested a review from a team as a code owner June 11, 2026 07:31
@rjan90 rjan90 self-assigned this Jun 11, 2026
@rjan90
rjan90 merged commit 6e019bc into filecoin-project:master Jun 11, 2026
5 of 6 checks passed
@BigLep

BigLep commented Jun 11, 2026

Copy link
Copy Markdown
Member

@rjan90 : there is a duplicate juliangruber in filecoin-pin that needs to be fixed. I tried but it's not letting me commit in github UI. I think fix that and then merge.

@rjan90

rjan90 commented Jun 11, 2026

Copy link
Copy Markdown
Contributor Author

Ah, did not see that before clicking. Will fix in a follow-up

@BigLep

BigLep commented Jun 11, 2026

Copy link
Copy Markdown
Member

Ah, did not see that before clicking. Will fix in a follow-up

Ack, ok. Oh and you merged which explains why I couldn't edit.

Yeah, terraform won't apply / work until we fix this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants